feat: monitoring app add/#677#678
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request expands the list of monitored applications by adding BLENDER, DATAGRIP, NOTION, FIGMA, and ROBLOX_STUDIO to the MonitoredApp enum, updating the corresponding database CHECK constraints across four tables, and adjusting the service tests. Feedback suggests replacing the redundant CHECK constraints with a dedicated reference table and foreign keys for better maintainability. Additionally, it is recommended to update the test to verify all MonitoredApp values using MonitoredApp.values() instead of manually listing them to ensure the test remains robust as the enum grows.
| OR app_id IN ( | ||
| 'VSCODE', | ||
| 'WEBSTORM', | ||
| 'INTELLIJ_IDEA', | ||
| 'PYCHARM', | ||
| 'GOLAND', | ||
| 'PHPSTORM', | ||
| 'RUBYMINE', | ||
| 'CLION', | ||
| 'RIDER', | ||
| 'ANDROID_STUDIO', | ||
| 'XCODE', | ||
| 'UNITY', | ||
| 'BLENDER', | ||
| 'DATAGRIP', | ||
| 'NOTION', | ||
| 'FIGMA', | ||
| 'ROBLOX_STUDIO' | ||
| ) |
There was a problem hiding this comment.
동일한 app_id 목록이 4개의 테이블 제약 조건(CHECK constraint)에 중복해서 작성되어 있습니다. 현재는 기존의 방식을 따르고 있으나, 모니터링 대상 앱이 추가될 때마다 모든 테이블의 제약 조건을 수동으로 업데이트해야 하므로 실수가 발생하기 쉽습니다. 장기적으로는 monitored_apps와 같은 별도의 참조 테이블을 생성하고 외래 키(FK)를 참조하는 방식으로 개선하는 것을 권장합니다.
| assertThat(result.apps()).contains( | ||
| MonitoredApp.VSCODE, | ||
| MonitoredApp.INTELLIJ_IDEA, | ||
| MonitoredApp.XCODE, | ||
| MonitoredApp.BLENDER, | ||
| MonitoredApp.DATAGRIP, | ||
| MonitoredApp.NOTION, | ||
| MonitoredApp.FIGMA, | ||
| MonitoredApp.ROBLOX_STUDIO | ||
| ); |
There was a problem hiding this comment.
현재 테스트 코드는 전체 모니터링 앱 중 일부만 수동으로 나열하여 검증하고 있습니다. MonitoredAppPolicy가 MonitoredApp.values()를 반환하도록 구현되어 있으므로, 테스트에서도 모든 enum 값이 포함되어 있는지 확인하는 것이 더 견고합니다. 이렇게 하면 향후 새로운 앱이 추가될 때마다 테스트 코드를 매번 수정하지 않아도 됩니다.
assertThat(result.apps()).containsExactlyInAnyOrder(MonitoredApp.values());
4 participants
경고: 이 기능을 개발한 사람은 모니터링 앱의 작동 방식을 모릅니다.
변경사항
관련 이슈
Closes #
추가 컨텍스트